Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Architecture overhaul #90

Merged
merged 3 commits into from
Mar 3, 2024

Conversation

Wbarillon
Copy link
Contributor

Légère révision de l'infrastructure du projet, afin de faciliter l'initialisation du projet et le développement de nouvelles fonctionnalités.

  • la base de données de développement est désormais disponible avec les données chargées via une image docker récupérable grâce à docker pull ;
  • l'application trawlwatcher comporte désormais un service dans le docker-compose.yaml, facilitant sa mise en place et son usage
  • les modifications étant principalement des modifications structurelles, l'intégrité du projet est maintenue : les autres workflows des développeurs déjà présents devraient être compatibles ;
  • l'unique modification à noter est le retrait du entrypoint.sh dans le Dockerfile de l'application. Ce dernier une copie de ce dernier, vers lequel point docker-compose.yaml est présente à la racine de bloom ;

@rv2931
Copy link
Collaborator

rv2931 commented Mar 3, 2024

Hello
comment tu gères les différents env dev/test/prod ?
Dans ma proposition (#85) je le gère avec une variable APP_ENV présente dans le .env.local et/ou via un export APP_ENV=dev|test|prod, tout le reste est plus ou moins automatique et basé sur sa valeur
De ton côté je n'ai pas compris, c'est un lien symbolique du .env ?
bonne idée de remonter le docker-compose.yml à la racine, car il faut effectivement que le .env soit dans le même répertoire (moi j'ai fini par faire un lien symbolique de docker-env/.env -> ../.env pour ne pas modifier la structure mais je pense que c'est plus propre et plus standard comme tu le propose)

Pour les histoire de lancer l'appli de la même manière en natif comme en docker, il faut toujours une gestion mixte config+variables d'environnement je pense, au final tout se passe dans le fichier bloom.config comme je le propose dans l'autre PR (#89). Je ferais une proposition à partir de ton PR si ton PR est validé

@RonanMorgan RonanMorgan merged commit db47f32 into dataforgoodfr:main Mar 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants